Skip to content

Changed symfony/web-server-bundle to dev package #8834

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Changed symfony/web-server-bundle to dev package #8834

wants to merge 1 commit into from

Conversation

xyNNN
Copy link
Contributor

@xyNNN xyNNN commented Dec 5, 2017

The symfony/web-server-bundle package should be installed as dev requirement. This bundle is not needable for production.

The symfony/web-server-bundle package should be installed as dev requirement. This bundle is not neeable for production.
Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xyNNN you are completely right! Thanks for fixing it and congrats on your first docs contribution.

Note to mergers: merge on 4.0 branch instead.

@xyNNN
Copy link
Contributor Author

xyNNN commented Dec 6, 2017

I'm happy to contribute 👍

@xabbuh xabbuh added this to the 4.0 milestone Dec 7, 2017
@xabbuh
Copy link
Member

xabbuh commented Dec 8, 2017

Thank you @xyNNN.

xabbuh added a commit that referenced this pull request Dec 8, 2017
This PR was submitted for the master branch but it was merged into the 4.0 branch instead (closes #8834).

Discussion
----------

Changed symfony/web-server-bundle to dev package

The symfony/web-server-bundle package should be installed as dev requirement. This bundle is not needable for production.

Commits
-------

72d2edb Changed symfony/web-server-bundle to dev package
@xabbuh xabbuh closed this Dec 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants